Skip to content

ci: upload coverage to codecov#10

Merged
silversurfer562 merged 1 commit into
mainfrom
feat/codecov-upload
May 25, 2026
Merged

ci: upload coverage to codecov#10
silversurfer562 merged 1 commit into
mainfrom
feat/codecov-upload

Conversation

@silversurfer562
Copy link
Copy Markdown
Member

Summary

Adds codecov upload step to the test workflow, mirroring the pattern in attune-ai. coverage.xml is already generated by pytest --cov and uploaded as a workflow artifact — this PR forwards it to codecov.io for trend tracking + missing-tests reports.

Required action before merge

CODECOV_TOKEN secret must be set on this repo first. codecov-action 6.0.1 rejects uploads to protected branches with HTTP 400 "Token required because branch is protected" if the token isn't present. The workflow stays green either way (fail_ci_if_error: false), but reports won't populate.

Steps:

  1. https://app.codecov.io/gh/Smart-AI-Memory/attune-help/settings → copy upload token
  2. https://github.com/Smart-AI-Memory/attune-help/settings/secrets/actions → new repo secret CODECOV_TOKEN

Diff

10 lines added in .github/workflows/tests.yml — the codecov upload step right after the existing artifact upload, same matrix-cell condition (ubuntu/3.11), same pin (v6.0.1 SHA-pinned per repo convention).

Test plan

  • Local edit + syntax-valid YAML
  • CI green on the PR
  • After merge + secret: first main run shows successful codecov upload

Adds the codecov upload step right after the existing
coverage.xml artifact upload. Same pattern as attune-ai
(codecov-action v6.0.1, fail_ci_if_error: false, token via env).

Requires CODECOV_TOKEN to be set as a GitHub Actions secret
on this repo BEFORE merging — without it, codecov-action 6.0.1
rejects uploads to protected branches with HTTP 400. The
workflow stays green either way (fail_ci_if_error: false), but
the codecov reports won't populate until the secret is in place.
@codecov
Copy link
Copy Markdown

codecov Bot commented May 25, 2026

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️

@silversurfer562 silversurfer562 merged commit 869c4c7 into main May 25, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant